Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix rendering of exported SVG #2794

Merged
merged 1 commit into from
Aug 8, 2017
Merged

Conversation

rndstr
Copy link
Contributor

@rndstr rndstr commented Aug 4, 2017

Replace invalid characters in attribute value; fixes the error

    Unescaped '<' not allowed in attributes values

Fixes #2534

There are a bunch of other errors when validating the SVG but renderers might be okay with these

element "div" undefined
element "span" undefined

@pidster would you mind opening the following SVG file in the browser where you get the other error and see whether it works now? weave-cloud-1501839338756.svg.zip

@rndstr rndstr requested a review from jpellizzari August 4, 2017 09:43
Replace invalid characters in attribute value; fixes the error

        Unescaped '<' not allowed in attributes values
@rndstr rndstr force-pushed the 2534-replace-invalid-id-characters branch from 8fabeaf to 998f92b Compare August 4, 2017 14:10
@rade
Copy link
Member

rade commented Aug 7, 2017

It was @2opremio that reported the problem, not @pidster.

The original report gives me exactly the shown error when opening in Chrome. Opening the above report works fine, so I think the fix works.

@rndstr
Copy link
Contributor Author

rndstr commented Aug 7, 2017

@rade thanks for confirming (the comment that reported it was by @pidster #2531 (comment))

Copy link
Contributor

@jpellizzari jpellizzari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rade rade mentioned this pull request Aug 7, 2017
@pidster
Copy link
Contributor

pidster commented Aug 8, 2017

LGTM

@rndstr rndstr merged commit 656fbac into master Aug 8, 2017
@rndstr rndstr deleted the 2534-replace-invalid-id-characters branch August 8, 2017 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SVG exporting is sometimes broken
4 participants